-
Notifications
You must be signed in to change notification settings - Fork 156
Conversation
@ElectricMaxxx @wouterj ship it boss 👍 |
@@ -30,6 +30,9 @@ the ``SeoMetadata``:: | |||
} | |||
} | |||
|
|||
If you are using PHP5.4+ you may also benefit from using the trait ``SeoAwareTrait`` | |||
to plug these behavior into your model. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you please put this into a tip directive?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is it .. tip:: ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes
Op 24 sep. 2014 08:30 schreef "Luis Cordova" notifications@github.com:
In bundles/seo/seo_aware.rst:
@@ -30,6 +30,9 @@ the
SeoMetadata
::
}
}+If you are using PHP5.4+ you may also benefit from using the trait
SeoAwareTrait
+to plug these behavior into your model.is it .. tip:: ?
—
Reply to this email directly or view it on GitHub
https://github.com/symfony-cmf/symfony-cmf-docs/pull/575/files#r17955772
.
👍 @wouterj |
Merged it into the correct branch now... Thanks @cordoval for your corrections, great to have people fixing the docs! |
This PR was submitted for the dev branch but it was merged into the master branch instead (closes #575). Discussion ---------- seo corrections |Q |A | |--- |--- | |Bug Fix? |n | |New Feature? |n | |BC Breaks? |n | |Deprecations?|n | |Tests Pass? |n | |Fixed Tickets|https://github.com/symfony-cmf/SeoBundle/issues/146| |License |MIT | |Doc PR | | Commits ------- b917a73 seo corrections f0d455e Fixed filename in comments
Sent using Gush